Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Mellanox]Implement plugins for PSU, fan and thermal #4041

Merged
merged 8 commits into from
Jan 24, 2020

Conversation

stephenxs
Copy link
Collaborator

@stephenxs stephenxs commented Jan 18, 2020

- What I did
Implement plugins for PSU with voltage, current and power supported
Implement plugins for fan.
Implement plugins for thermal, including get_temperature, get_high_threshold and get_critical_highthreshold.
This PR depends on:

  1. [sonic_fan]add constant for fan direction #76
  2. [sonic_thermal]Add abstract class for thermal plugin #75

- How I did it

- How to verify it
Manually test all the interfaces.
fan-psu.test.zip
fan-psu.testresult.txt
testresult.thermal.txt
thermal.plugin.test.zip

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@jleveque
Copy link
Contributor

Retest broadcom please

@jleveque
Copy link
Contributor

Retest vsimage please

@jleveque jleveque merged commit 3239d7f into sonic-net:master Jan 24, 2020
@stephenxs stephenxs deleted the plugins branch January 26, 2020 07:53
stephenxs added a commit to stephenxs/sonic-buildimage that referenced this pull request Mar 5, 2020
* [plugins]add fan functions, add voltage, current, power for psu
* [plugins]link fanutil.py and psuutil.py to those in 2700
* [plugin]add thermal
* [plugin]add symbol links for thermalutil for all SKUs

Conflicts:
	device/mellanox/x86_64-mlnx_msn2410-r0/plugins/psuutil.py
	device/mellanox/x86_64-mlnx_msn2700-r0/plugins/psuutil.py
	device/mellanox/x86_64-mlnx_msn2740-r0/plugins/psuutil.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants